Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(aws): add multi-region concept page #3519

Merged
merged 9 commits into from
Mar 28, 2024
Merged

chore(aws): add multi-region concept page #3519

merged 9 commits into from
Mar 28, 2024

Conversation

Langleu
Copy link
Member

@Langleu Langleu commented Mar 25, 2024

Description

Adds a concept page on the dual-region topic, elaborating on limitations, requirements, and an outline of the architecture.

npm run start

navigate to : http://localhost:3000/docs/next/self-managed/concepts/multi-region/dual-region/

When should this change go live?

  • This change is not yet live and should not be merged until {ADD_DATE} (apply hold label or convert to draft PR)?
  • There is no urgency with this change.
  • This change or page is part of a marketing blog, conference talk, or something else on a schedule.
  • This functionality is already available but undocumented.
  • This is a bug fix or security concern.

PR Checklist

  • I have added changes to the relevant /versioned_docs directory, or they are not for an already released version.
  • I have added changes to the main /docs directory (aka /next/), or they are not for future versions.
  • My changes require an Engineering review, and I've assigned an engineering manager or tech lead as a reviewer, or my changes do not require an Engineering review.
  • My changes require a technical writer review, and I've assigned @christinaausley as a reviewer, or my changes do not require a technical writer review.

@Langleu Langleu added component:self-managed Docs and issues related to Camunda Platform 8 Self-Managed target:8.5 component:infex Issues related with InfEx project labels Mar 25, 2024
@Langleu Langleu requested a review from maxdanilov March 25, 2024 15:09
@Langleu Langleu self-assigned this Mar 25, 2024
Copy link
Contributor

github-actions bot commented Mar 25, 2024

👋 🤖 🤔 Hello! Did you make your changes in all the right places?

These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.4/.

  • docs/self-managed/concepts/multi-region/dual-region.md
  • docs/self-managed/concepts/multi-region/img/dual-region.svg

You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines.

Copy link
Member

@maxdanilov maxdanilov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, leaving some feedback.

docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
@Langleu
Copy link
Member Author

Langleu commented Mar 26, 2024

@maxdanilov implemented the PR suggestions in ecb1d86

Copy link
Member

@maxdanilov maxdanilov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, mainly leaving wording suggestions

docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
docs/self-managed/concepts/multi-region/dual-region.md Outdated Show resolved Hide resolved
@Langleu Langleu changed the title DRAFT: chore(aws): add multi-region concept page [DRAFT] chore(aws): add multi-region concept page Mar 27, 2024
@Langleu Langleu changed the title [DRAFT] chore(aws): add multi-region concept page chore(aws): add multi-region concept page Mar 27, 2024
@Langleu Langleu marked this pull request as ready for review March 27, 2024 10:11
@Langleu Langleu requested a review from christinaausley March 27, 2024 10:11
@maxdanilov maxdanilov self-requested a review March 27, 2024 10:12
Copy link
Member

@maxdanilov maxdanilov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@Langleu
Copy link
Member Author

Langleu commented Mar 27, 2024

@christinaausley , I'm ready for a final review 🔥

Copy link
Contributor

@christinaausley christinaausley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My standard review is complete, though I'll have another look over this before we merge [infex-89](https://github.com/camunda/camunda-docs/tree/infex-89) into main.

@Langleu Langleu merged commit 95092d4 into infex-89 Mar 28, 2024
5 of 6 checks passed
@Langleu Langleu deleted the infex-97 branch March 28, 2024 07:41
Langleu added a commit that referenced this pull request Apr 4, 2024
* docs(dual-region): add concept page

* chore(aws): add concept page

* chore(aws): dual-region, add region loss info

* docs(dual-region): implement PR suggestions for concept page

* docs(dual-region): add look at region loss for concept page

* docs(dual-region): small restructure

* docs(dual-region): add missing info on sizing and lost data

* docs(dual-region): more concept page adjustments

* style(formatting): technical review

---------

Co-authored-by: Christina Ausley <[email protected]>
Langleu added a commit that referenced this pull request Apr 5, 2024
* docs(dual-region): add concept page

* chore(aws): add concept page

* chore(aws): dual-region, add region loss info

* docs(dual-region): implement PR suggestions for concept page

* docs(dual-region): add look at region loss for concept page

* docs(dual-region): small restructure

* docs(dual-region): add missing info on sizing and lost data

* docs(dual-region): more concept page adjustments

* style(formatting): technical review

---------

Co-authored-by: Christina Ausley <[email protected]>
Langleu added a commit that referenced this pull request Apr 5, 2024
* docs(aws): dual-region setup in AWS (#3430)

* docs(aws): add dual-region setup

* docs(aws): additions to the dual-region setup

* docs(aws): switch to tabs and collapsables for dual-region

* docs(aws): further adjustments to the dual-region

* style(formatting): initial technical review

* docs(aws): pull request adjustments

pre-cloning setup

* docs(aws): dual-region switch to git clone

* docs(aws): pull request adjustments

* docs(aws): remove tabs and simplify with more scripts

* docs(aws): use new folder path

* docs(aws): implement pr suggestions

* small tweak

---------

Co-authored-by: Christina Ausley <[email protected]>

* chore(aws): add multi-region concept page (#3519)

* docs(dual-region): add concept page

* chore(aws): add concept page

* chore(aws): dual-region, add region loss info

* docs(dual-region): implement PR suggestions for concept page

* docs(dual-region): add look at region loss for concept page

* docs(dual-region): small restructure

* docs(dual-region): add missing info on sizing and lost data

* docs(dual-region): more concept page adjustments

* style(formatting): technical review

---------

Co-authored-by: Christina Ausley <[email protected]>

* docs(dual-region): add disaster recovery RPO and RTO

* docs(dual-regeion): address review feedback of concept page

* docs(dual-region): add operational procedure (#3473)

* docs(dual-region): add operational procedure

* docs(dual-region): first rework of the operational procedure

* docs(multi-region): describe all steps for operational guide

* docs(dual-region): adjust viewbox and add more verification steps

* docs(dual-region): apply prettier suggestion

* docs(dual-region): move helm chart release and version to env var

* docs(dual-region): adjust missing links by resolving todos

* docs(dual-region): add remaining verifcation steps

* docs(dual-region): add info admonition on elasticsearch backup source

* docs(dual-region): address review feedback first batch
including changes in wording and capitalization

* docs(dual-region): change use of environment variables

* docs(dual-region): align disclaimer with concept page

* docs(dual-region): add termonology and tabs for env preqs

* docs(dual-region): added simplifcations and more verifications

* docs(dual-region): fix typo in verification

* docs(dual-region): fix todo linkage

* style(formatting): technical review

* docs(dual-region): minor kubectl refinements

added of image source

---------

Co-authored-by: Christina Ausley <[email protected]>
theburi pushed a commit that referenced this pull request Jun 5, 2024
* docs(aws): dual-region setup in AWS (#3430)

* docs(aws): add dual-region setup

* docs(aws): additions to the dual-region setup

* docs(aws): switch to tabs and collapsables for dual-region

* docs(aws): further adjustments to the dual-region

* style(formatting): initial technical review

* docs(aws): pull request adjustments

pre-cloning setup

* docs(aws): dual-region switch to git clone

* docs(aws): pull request adjustments

* docs(aws): remove tabs and simplify with more scripts

* docs(aws): use new folder path

* docs(aws): implement pr suggestions

* small tweak

---------

Co-authored-by: Christina Ausley <[email protected]>

* chore(aws): add multi-region concept page (#3519)

* docs(dual-region): add concept page

* chore(aws): add concept page

* chore(aws): dual-region, add region loss info

* docs(dual-region): implement PR suggestions for concept page

* docs(dual-region): add look at region loss for concept page

* docs(dual-region): small restructure

* docs(dual-region): add missing info on sizing and lost data

* docs(dual-region): more concept page adjustments

* style(formatting): technical review

---------

Co-authored-by: Christina Ausley <[email protected]>

* docs(dual-region): add disaster recovery RPO and RTO

* docs(dual-regeion): address review feedback of concept page

* docs(dual-region): add operational procedure (#3473)

* docs(dual-region): add operational procedure

* docs(dual-region): first rework of the operational procedure

* docs(multi-region): describe all steps for operational guide

* docs(dual-region): adjust viewbox and add more verification steps

* docs(dual-region): apply prettier suggestion

* docs(dual-region): move helm chart release and version to env var

* docs(dual-region): adjust missing links by resolving todos

* docs(dual-region): add remaining verifcation steps

* docs(dual-region): add info admonition on elasticsearch backup source

* docs(dual-region): address review feedback first batch
including changes in wording and capitalization

* docs(dual-region): change use of environment variables

* docs(dual-region): align disclaimer with concept page

* docs(dual-region): add termonology and tabs for env preqs

* docs(dual-region): added simplifcations and more verifications

* docs(dual-region): fix typo in verification

* docs(dual-region): fix todo linkage

* style(formatting): technical review

* docs(dual-region): minor kubectl refinements

added of image source

---------

Co-authored-by: Christina Ausley <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:infex Issues related with InfEx project component:self-managed Docs and issues related to Camunda Platform 8 Self-Managed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants